Skip to content

Cleanup prettier references#2781

Merged
rajsite merged 2 commits intomainfrom
cleaup-prettier
Dec 6, 2025
Merged

Cleanup prettier references#2781
rajsite merged 2 commits intomainfrom
cleaup-prettier

Conversation

@rajsite
Copy link
Copy Markdown
Member

@rajsite rajsite commented Dec 6, 2025

Pull Request

🤨 Rationale

Removes inline pretter disable statements and updates docs to reference eslint

Follow-up to #2773

👩‍💻 Implementation

See above

🧪 Testing

Rely on CI

✅ Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed.

@rajsite rajsite marked this pull request as ready for review December 6, 2025 00:27
@rajsite
Copy link
Copy Markdown
Member Author

rajsite commented Dec 6, 2025

@jattasNI bypass merging for very mechanical change just removing comments. If feedback can address in follow-up

@rajsite rajsite merged commit aabdf2f into main Dec 6, 2025
11 checks passed
@rajsite rajsite deleted the cleaup-prettier branch December 6, 2025 00:29
@rajsite rajsite mentioned this pull request Dec 16, 2025
1 task
rajsite added a commit that referenced this pull request Dec 16, 2025
# Pull Request

## 🤨 Rationale

- Missed some prettier config files in #2781

## 👩‍💻 Implementation

- Removed the prettier config files
- Kept the prettierignore in the root but configured it to ignore all
files to prevent accidental formatting in repo

## 🧪 Testing

Rely on CI

## ✅ Checklist

- [x] I have updated the project documentation to reflect my changes or
determined no changes are needed.

---------

Co-authored-by: rajsite <1588923+rajsite@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant